Skip to content

Conversation

@JakobKlotz
Copy link
Contributor

What?

Expanded the with-docker example for usage with bun and added build instructions to the README.

Why?

The existing example supports npm, yarn and pnpm, but not bun.

How?

  • Add a separate Dockerfile.bun placed next to the existing Dockerfile, which is more sensible for the existing solution.
  • Dockerfile.bun uses the official oven/bun base image (different runtime/tooling than node:18-alpine), so a separate file should be clearer.
  • Existing Docker commands in the README remain unchanged, because Docker still uses the existing Dockerfile by default.
  • The bun example can be built with: docker build -f Dockerfile.bun -t nextjs-docker .

Related to #78465, #82998

@ijjk ijjk added the examples Issue was opened via the examples template. label Aug 26, 2025
@ijjk
Copy link
Member

ijjk commented Aug 26, 2025

Allow CI Workflow Run

  • approve CI run for commit: 6cae383

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@icyJoseph
Copy link
Collaborator

Hi, could you try to address the bot comments?

@JakobKlotz
Copy link
Contributor Author

Hi, could you try to address the bot comments?

Sure, addressed them and left some comments for clarification!

Existing docker example within the same directory does not create a next.config.js for standalone output (within the Dockerfile), hence it's now more consistent.

Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
@icyJoseph icyJoseph enabled auto-merge (squash) September 3, 2025 21:54
@icyJoseph icyJoseph merged commit f41c809 into vercel:canary Sep 3, 2025
63 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

examples Issue was opened via the examples template. locked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants